Skip to content

Platform 2.0 Infra and Reboot Reason support for Dell Platforms#3307

Closed
sridhar-ravindran wants to merge 7 commits intosonic-net:201904from
sridhar-ravindran:s6000_reboot_reason
Closed

Platform 2.0 Infra and Reboot Reason support for Dell Platforms#3307
sridhar-ravindran wants to merge 7 commits intosonic-net:201904from
sridhar-ravindran:s6000_reboot_reason

Conversation

@sridhar-ravindran
Copy link
Contributor

@sridhar-ravindran sridhar-ravindran commented Aug 8, 2019

- What I did
Ported Platform 2.0 Infra and Dell S6100,Z9100,S6000 Changes from master to 201904 branch
- How I did it
git cherry-pick and resolve conflicts
It has 3 porting
a) Platform 2.0 base infra + Z9100,S6100 Reboot Reason
b) S6000 Directory clean up
c) S6000 Reboot Reason support
- How to verify it
Currently unable to verify because linux-kernel build is failing due to http timeout
- Description for the changelog

Platform 2.0 Infra and Reboot Reason support for Dell Platforms

- A picture of a cute animal (not mandatory but encouraged)

sridhar-ravindran and others added 3 commits August 5, 2019 01:46
@sridhar-ravindran
[devices]: Porting DELL Platform 2.0 API Infra and Reboot Reason supp…
e9ce93a 
…ort to 201904
@gengankarthik @sridhar-ravindran
Porting S6000 Directory Move PR to 201911
4eef2d2
@sridhar-ravindran
[DELLEMC][S6000] Platform 2.0 Reboot Reason Fix in S6000
2f4a509 
Added Reboot Reason for S6000 in platform 2.0
Fixed issue in process-reboot-cause
Added package uninstall code in platform de-init code for z9100, s6100
- How I did it
-> Added support for S6000 Reboot Reason
-> Added platform.py for all platforms
-> Verified show reboot-cause command with the code changes. Added UT logs with show reboot-cause
-> Modified process-reboot-cause service to start after pmon.service. In S6000, we have to wait for nvram to be loaded.
-> If reboot-cause service starts before pmon.service, show reboot-cause is showing incorrect reason.
-> Bug fix in process-reboot-cause file
- import sonic_platform
+ import sonic_platform.platform
@jleveque
Copy link
Contributor

jleveque commented Aug 8, 2019

Retest this please

2 similar comments
@jleveque
Copy link
Contributor

jleveque commented Aug 8, 2019

Retest this please

@jleveque
Copy link
Contributor

jleveque commented Aug 23, 2019

Retest this please

@sridhar-ravindran
Copy link
Contributor Author

sridhar-ravindran commented Sep 3, 2019

Unable to compile build because of following error:-

4.9.168-1+deb9u3_all.deb /sonic/target/debs/stretch/
--2019-09-03 10:14:21-- http://security.debian.org/debian-security/pool/updates/main/l/linux/linux_4.9.168-1+deb9u3.dsc
Resolving security.debian.org (security.debian.org)... 128.61.240.73, 128.31.0.63, 128.101.240.215, ...
Connecting to security.debian.org (security.debian.org)|128.61.240.73|:80... connected.
HTTP request sent, awaiting response... 302 Found
Location: http://security-cdn.debian.org/debian-security/pool/updates/main/l/linux/linux_4.9.168-1+deb9u3.dsc [following]
--2019-09-03 10:14:21-- http://security-cdn.debian.org/debian-security/pool/updates/main/l/linux/linux_4.9.168-1+deb9u3.dsc
Resolving security-cdn.debian.org (security-cdn.debian.org)... 151.101.40.204, 2a04:4e42:a::204
Connecting to security-cdn.debian.org (security-cdn.debian.org)|151.101.40.204|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2019-09-03 10:14:21 ERROR 404: Not Found.

@jleveque
Copy link
Contributor

jleveque commented Sep 5, 2019

Retest this please

@sridhar-ravindran
Copy link
Contributor Author

sridhar-ravindran commented Nov 22, 2019

Hi,
We have raised #3767 to support this feature in master.
Once approved, will raise a new PR for this branch. Request you to review the same.

@sridhar-ravindran sridhar-ravindran deleted the s6000_reboot_reason branch November 22, 2019 05:36
mssonicbld added a commit that referenced this pull request Nov 20, 2024
@mssonicbld
[submodule] Update submodule sonic-swss to the latest HEAD automatica…
71fd069 
…lly (#20853)

#### Why I did it
src/sonic-swss
```
* 1843de4 - (HEAD -> master, origin/master, origin/HEAD) Ensure second route group OID is correct (#3380) (2 hours ago) [Lawrence Lee]
* c86efa1 - Added support for "UNDERLAY_SET_DSCP" and "UNDERLAY_SET_DSCPV6" tables (#3307) (2 hours ago) [siqbal1986]
* 0b331f0 - platform/innovium renaming to platform/marvell-teralynx (#3252) (10 hours ago) [krismarvell]
* c865803 - [mirror] erspan ipv6 underlay (see PR #1817) (#3317) (30 hours ago) [mramezani95]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jleveque jleveque

Labels

Enhancement ➕ Platform: DellEMC

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sridhar-ravindran @jleveque @gengankarthik